.github/pull_request_template.md (1)

1-4: Jira 의존성 제거는 좋습니다. GitHub 이슈 자동 연결 문구를 안내로 추가 제안

조직 도구에 중립적인 템플릿으로 간결해졌습니다. 다만 기여자들이 PR 머지 시 이슈가 자동으로 닫히도록 “Closes #번호” 형태의 안내를 한 줄 넣어두면 실수 방지에 도움 됩니다.

적용 예시(해당 블록 중간에 한 줄 안내 추가):

 # 🚩 연관 이슈
+
+- GitHub 이슈 예: Closes #123

 # 🔂 변경 내역

gss-common/src/main/java/com/devoops/exception/errorcode/ErrorCode.java (1)

9-11: 이름 충돌(혼동) 가능성: NOT_FOUND 네이밍이 4xx/5xx에 혼재

REGISTRY_GITHUB_REPOSITORY_NOT_FOUND(400)와 GITHUB_REPOSITORY_NOT_FOUND(500)이 공존해 의미가 모호해질 수 있습니다. 현재 PR 의도는 “GitHub 404를 400으로 변환”인데, 식별자에 NOT_FOUND가 들어가면 404를 연상시키는 여지가 있습니다. 운영상 혼선을 줄이려면:

• 주석/Javadoc로 용례를 명확히 기록, 또는
• REGISTRY_GITHUB_REPOSITORY_REGISTRATION_FAILED처럼 결과(등록 실패) 중심 네이밍 고려

메시지는 그대로 두고 식별자만 구체화하는 것도 방법입니다.

Also applies to: 29-29

gss-api-app/src/main/java/com/devoops/service/facade/RepositoryFacadeService.java (1)

34-45: 트랜잭션 경계 내 외부 I/O 호출 최소화 제안

현재 save 트랜잭션 내에서 웹훅 등록(외부 네트워크 I/O)과 이벤트 발행이 수행됩니다. 타임아웃/재시도 시 장기 트랜잭션이 될 수 있어 DB 락 유지 시간이 늘어날 우려가 있습니다. 선택지:

• 웹훅 등록을 트랜잭션 밖에서 수행(실패 시 보상/정리 로직 포함)
• outbox/event-relay 패턴 도입으로 이벤트 발행 비동기화

gss-api-app/src/test/java/com/devoops/service/facade/RepositoryFacadeServiceTest.java (3)

94-102: 불필요한 지역변수 제거

mockingErrorWhenCreateWebHook 내 WebHookCreateResponse 로컬 변수는 사용되지 않습니다. 제거해 주세요.

적용 diff:

         private void mockingErrorWhenCreateWebHook() {
             GithubRepoInfoResponse mockResponse = new GithubRepoInfoResponse(123, "testName", "testUrl",
                     new OwnerResponse("김건우"));
-            WebHookCreateResponse mockWebHookCreateResponse = new WebHookCreateResponse(123);
             Mockito.when(gitHubClient.getRepositoryInfo(anyString(), anyString(), anyString()))
                     .thenReturn(mockResponse);
             Mockito.when(gitHubClient.createWebhook(any(), any(), any(), any()))
                     .thenThrow(new GithubNotFoundException("mocking error"));
         }

---

60-69: 레포지토리 정보 조회 단계 실패 케이스 추가 테스트 제안

현재는 “웹훅 등록” 단계의 404만 커버합니다. GitHub 레포지토리 정보 조회(getRepositoryInfo) 단계에서의 404도 동일하게 변환되는지 보강 테스트를 권장합니다.

예시 테스트(새 메서드 추가):

@Test
void 레포지토리_정보_조회_404_시_애플리케이션_에러로_전환한다() {
    User user = userGenerator.generate("김건우");
    RepositorySaveRequest request = new RepositorySaveRequest("https://github.com/octocat/Hello-World");

    // getRepositoryInfo가 404를 유발하도록 설정
    Mockito.when(gitHubClient.getRepositoryInfo(anyString(), anyString(), anyString()))
           .thenThrow(new GithubNotFoundException("mocking error"));

    assertThatThrownBy(() -> repositoryFacadeService.save(request, user))
        .isInstanceOf(GssException.class)
        // 가능하면 에러코드 단정으로 교체 (앞선 코멘트 참고)
        .hasMessage(ErrorCode.REGISTRY_GITHUB_REPOSITORY_NOT_FOUND.getMessage());

    // 단락 평가 검증: 웹훅 생성은 시도되지 않아야 함
    Mockito.verify(gitHubClient, times(0)).createWebhook(any(), any(), any(), any());
}

원하시면 제가 바로 추가 PR 패치까지 준비하겠습니다.

---

60-69: 예외 메시지 대신 에러 코드로 단정 검증 권장

현재 테스트는 메시지 문자열(.hasMessage(...))에 의존해 있어, 국제화나 문구 변경 시 깨질 위험이 있습니다.
GssException에는 Lombok @Getter로 생성된 getErrorCode() 접근자가 있으므로, 메시지 대신 에러 코드를 검증하도록 리팩토링하는 것이 더 견고합니다.

예시 변경안:

assertThatThrownBy(() -> repositoryFacadeService.save(request, user))
    .isInstanceOf(GssException.class)
    .satisfies(ex -> assertThat(((GssException) ex).getErrorCode())
        .isEqualTo(ErrorCode.REGISTRY_GITHUB_REPOSITORY_NOT_FOUND));

• 파일: gss-api-app/src/test/java/com/devoops/service/facade/RepositoryFacadeServiceTest.java
• 위치: 테스트 메소드 내 assertThatThrownBy 호출부

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.